Your change would make Zydis to initialize all the time, wich is something we do not want. It should initialize only when needed.

wich is something we do not want.

Why?

wich is something we do not want.

Why?

We don't need to load stuffs we don't use. Why having more memory consumed for nothing?

Only if you enabled HAVE_TRACE when compiling, which non-devs should not enable.

Still, I don't want lib loaded if we don't use it. Adding a check of dump or trace enable is easy enough, I don't see why we shouldn't keep the test.

My intention is that I think it is not intuitive to use BOX64_TRACE=0-1 to enable Zydis loading. Maybe a new option can be added for control, and the new option will be automatically turned on when BOX64_TRACE is on.

My intention is that I think it is not intuitive to use BOX64_TRACE=0-1 to enable Zydis loading. Maybe a new option can be added for control, and the new option will be automatically turned on when BOX64_TRACE is on.

I understand, and I aggree but I think there is a need for a env. var. option for that.

Ok, it's nice, but it's either missing something, or the title of the is wrong now.

Ok, it's nice, but it's either missing something, or the title of the is wrong now.

Sorry I forgot to change the title. Now it is the same as the commit message.

You should update the commit message as well as PR title to reflect the name change.

You should update the commit message as well as PR title to reflect the name change.

Sorry, I forgot again... Fixed.

Sorry for missing changes again and again...